Skip to content

chore: Rename isIntegratedGPUID function to isOrinGPUID#1674

Merged
elezar merged 2 commits intoNVIDIA:mainfrom
elezar:improve-lib-csv-readability
Mar 4, 2026
Merged

chore: Rename isIntegratedGPUID function to isOrinGPUID#1674
elezar merged 2 commits intoNVIDIA:mainfrom
elezar:improve-lib-csv-readability

Conversation

@elezar
Copy link
Member

@elezar elezar commented Feb 23, 2026

@coveralls
Copy link

coveralls commented Feb 23, 2026

Pull Request Test Coverage Report for Build 22625083068

Details

  • 4 of 5 (80.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 42.799%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/nvcdi/lib-csv.go 4 5 80.0%
Totals Coverage Status
Change from base Build 22589959798: 0.002%
Covered Lines: 6342
Relevant Lines: 14818

💛 - Coveralls

@elezar elezar force-pushed the improve-lib-csv-readability branch from 685d2df to 4f7a404 Compare February 25, 2026 11:11
@elezar elezar changed the title chore: Add comment to isIntegratedGPUID function chore: Rename isIntegratedGPUID function to isOrinGPUID Feb 25, 2026
// We explicitly handle this case.
if pciInfo.Domain == 0 && pciInfo.Bus == 1 && pciInfo.Device == 0 {
return true, nil
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to delete the lines that immediately proceed this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. Trying to do too many things.

@elezar elezar force-pushed the improve-lib-csv-readability branch 2 times, most recently from 63a2e81 to 6acc956 Compare March 3, 2026 13:26
elezar added 2 commits March 3, 2026 14:26
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the improve-lib-csv-readability branch from 6acc956 to fc8606c Compare March 3, 2026 13:26
@elezar elezar requested a review from cdesiniotis March 3, 2026 14:55
@elezar elezar merged commit 88da3d2 into NVIDIA:main Mar 4, 2026
16 checks passed
@elezar elezar deleted the improve-lib-csv-readability branch March 4, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants